@TomasVotruba ready 👍

Seems good to me. The leaveNode() in AbstractRector should be marked as final to avoid override. Not just docblock, but natively.
Just in case someone would try to use it, so it's not skipped silent.y

The leaveNode already removed from AbstractRector, do you mean I need to add leaveNode method back to AbstractRector ?

Yes, return it empty and mark it final

I just checked that the leaveNode() method was already final, so it already final for a while, I think removing leaveNode is fine.

Yes, but if we remove the final now, people will be able to use it (incorrectly).

sure, i added back leaveNode ac80ca5 👍

Thanks 🙏